Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add QOS and buffer profiles for SKU Arista-7800R3-48CQM2-C48 #10752

Merged
merged 2 commits into from
May 23, 2022

Conversation

ysmanman
Copy link
Contributor

@ysmanman ysmanman commented May 5, 2022

Why I did it

QOS and buffer profiles are missing for Arista SKU Arista-7800R3-48CQM2-C48.

How I did it

Created a QOS and buffer profiles for Arista-7800R3-48CQM2-C48. The profiles are the same of those for Arista-7800R3-48CQ2-C48 because the forwarding ASICs are the same generation in both SKUs.

How to verify it

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@arlakshm
Copy link
Contributor

arlakshm commented May 5, 2022

Hi @ysmanman, Thanks for adding this. Can you please add unit test for this SKU as well, like the one added for arista7800r3-48cq2-lc

@arlakshm arlakshm self-requested a review May 5, 2022 22:52
@ysmanman ysmanman requested a review from lguohan as a code owner May 6, 2022 01:27
@ysmanman
Copy link
Contributor Author

ysmanman commented May 6, 2022

Hi @ysmanman, Thanks for adding this. Can you please add unit test for this SKU as well, like the one added for arista7800r3-48cq2-lc

Hi @arlakshm, I added unit test for the sku. Please take a look.

@neethajohn
Copy link
Contributor

@ysmanman, please update the PR description

@ysmanman ysmanman changed the title Add QOS and buffer profiles for Arista SKU. Add QOS and buffer profiles for SKU Arista-7800R3-48CQM2-C48 May 12, 2022
@ysmanman
Copy link
Contributor Author

@ysmanman, please update the PR description

@neethajohn Updated PR description.

@arlakshm arlakshm merged commit b23ad67 into sonic-net:master May 23, 2022
liushilongbuaa pushed a commit to liushilongbuaa/sonic-buildimage that referenced this pull request Jun 20, 2022
…anch

Related work items: #52, #71, #73, #75, #77, sonic-net#1306, sonic-net#1588, sonic-net#1991, sonic-net#2031, sonic-net#2040, sonic-net#2053, sonic-net#2066, sonic-net#2069, sonic-net#2087, sonic-net#2107, sonic-net#2110, sonic-net#2112, sonic-net#2113, sonic-net#2117, sonic-net#2124, sonic-net#2125, sonic-net#2126, sonic-net#2128, sonic-net#2130, sonic-net#2131, sonic-net#2132, sonic-net#2133, sonic-net#2134, sonic-net#2135, sonic-net#2136, sonic-net#2137, sonic-net#2138, sonic-net#2139, sonic-net#2140, sonic-net#2143, sonic-net#2158, sonic-net#2161, sonic-net#2233, sonic-net#2243, sonic-net#2250, sonic-net#2254, sonic-net#2260, sonic-net#2261, sonic-net#2267, sonic-net#2278, sonic-net#2282, sonic-net#2285, sonic-net#2288, sonic-net#2289, sonic-net#2292, sonic-net#2294, sonic-net#8887, sonic-net#9279, sonic-net#9390, sonic-net#9511, sonic-net#9700, sonic-net#10025, sonic-net#10322, sonic-net#10479, sonic-net#10484, sonic-net#10493, sonic-net#10500, sonic-net#10580, sonic-net#10595, sonic-net#10628, sonic-net#10634, sonic-net#10635, sonic-net#10644, sonic-net#10670, sonic-net#10691, sonic-net#10716, sonic-net#10731, sonic-net#10750, sonic-net#10751, sonic-net#10752, sonic-net#10761, sonic-net#10769, sonic-net#10775, sonic-net#10776, sonic-net#10779, sonic-net#10786, sonic-net#10792, sonic-net#10793, sonic-net#10800, sonic-net#10806, sonic-net#10826, sonic-net#10839, sonic-net#10840, sonic-net#10842, sonic-net#10844, sonic-net#10847, sonic-net#10849, sonic-net#10852, sonic-net#10865, sonic-net#10872, sonic-net#10877, sonic-net#10886, sonic-net#10889, sonic-net#10903, sonic-net#10904, sonic-net#10905, sonic-net#10913, sonic-net#10914, sonic-net#10916, sonic-net#10919, sonic-net#10925, sonic-net#10926, sonic-net#10929, sonic-net#10933, sonic-net#10934, sonic-net#10937, sonic-net#10941, sonic-net#10947, sonic-net#10952, sonic-net#10953, sonic-net#10957, sonic-net#10959, sonic-net#10971, sonic-net#10972, sonic-net#10980
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants